From 6174a8625a5d727d64a5111b4a6b2a3280ef8db4 Mon Sep 17 00:00:00 2001 From: Keir Fraser Date: Fri, 18 Jul 2008 11:23:25 +0100 Subject: [PATCH] X86: Fix cpufreq _psd HW_ALL coordination bug Currently xen cpufreq has a bug when handleing _psd HW_ALL, which will result in system broken when _psd HW_ALL. This patch fix this bug by handling _psd HW_ALL in same way as SW_ALL coordiantion, for the seek of safety. Signed-off-by: Liu Jinsong --- xen/arch/x86/acpi/cpufreq/cpufreq.c | 15 ++++++--------- 1 file changed, 6 insertions(+), 9 deletions(-) diff --git a/xen/arch/x86/acpi/cpufreq/cpufreq.c b/xen/arch/x86/acpi/cpufreq/cpufreq.c index b3c02279d0..cabe94ea93 100644 --- a/xen/arch/x86/acpi/cpufreq/cpufreq.c +++ b/xen/arch/x86/acpi/cpufreq/cpufreq.c @@ -433,16 +433,13 @@ acpi_cpufreq_cpu_init(struct cpufreq_policy *policy) perf = data->acpi_data; policy->shared_type = perf->shared_type; - /* - * Will let policy->cpus know about dependency only when software - * coordination is required. + /* + * Currently the latest linux (kernel version 2.6.26) + * still has issue when handle the situation _psd HW_ALL coordination. + * In Xen hypervisor, we handle _psd HW_ALL coordination in same way as + * _psd SW_ALL coordination for the seek of safety. */ - if (policy->shared_type == CPUFREQ_SHARED_TYPE_ALL || - policy->shared_type == CPUFREQ_SHARED_TYPE_ANY) { - policy->cpus = perf->shared_cpu_map; - } else { - policy->cpus = cpumask_of_cpu(cpu); - } + policy->cpus = perf->shared_cpu_map; /* capability check */ if (perf->state_count <= 1) { -- 2.30.2